Skip to content

Conversation

@wpalani
Copy link
Member

@wpalani wpalani commented Nov 20, 2025

Proposed changes

This PR removes the language selector (locale) field from the IntakeStep and the react app in general. Instead the locale will be determined based on WordPress locale and the backend will handle the logic of fetching the active locale and attaching it to the AI service when needed.

As of this moment, the FE doesn't have any use for the locale however for any future use cases, this PR adds a getLocale selector to the store's runtime slice, should the FE need to make decision based on the locale.

Updated UI

CleanShot 2025-11-20 at 16 55 17@2x

Internal

https://jira.newfold.com/browse/PRESS0-3328

Type of Change

Production

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Dependency update
  • Refactoring / housekeeping (changes to files not directly related to functionality)

Development

  • Tests
  • Dependency update
  • Environment update / refactoring
  • Documentation Update

Visual

Checklist

  • I have read the CONTRIBUTING doc
  • I have viewed my change in a web-browser
  • Linting and tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)

Further comments

@github-actions
Copy link
Contributor

Copy link
Contributor

@kreamweb kreamweb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested in local, it works fine

Copy link
Contributor

@francgrasso francgrasso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested locally. Nothing found in debug log or in laravel log.
Configured my wp install to be Italian and the whole went in Italian too. But onboarding panels where in English, I probably need translation files?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants